Hierarchy #288

FelixFrizzy · 2025-10-27T08:54:46Z

Summary by CodeRabbit

New Features
- Vocabularies now display top concepts by default, improving concept discovery and navigation.
Chores
- Updated development dependencies to stable releases.

Bumps [send](https://github.com/pillarjs/send) to 0.19.1 and updates ancestor dependency [browser-sync](https://github.com/BrowserSync/browser-sync). These dependencies need to be updated together. Updates `send` from 0.16.2 to 0.19.1 - [Release notes](https://github.com/pillarjs/send/releases) - [Changelog](https://github.com/pillarjs/send/blob/master/HISTORY.md) - [Commits](https://github.com/pillarjs/send/commits) Updates `browser-sync` from 2.29.3 to 3.0.3 - [Release notes](https://github.com/BrowserSync/browser-sync/releases) - [Changelog](https://github.com/BrowserSync/browser-sync/blob/master/changelog.js) - [Commits](BrowserSync/browser-sync@v2.29.3...v3.0.3) --- updated-dependencies: - dependency-name: send dependency-type: indirect - dependency-name: browser-sync dependency-type: direct:development ... Signed-off-by: dependabot[bot] <[email protected]>

Bumps [serve-static](https://github.com/expressjs/serve-static) to 1.16.2 and updates ancestor dependency [browser-sync](https://github.com/BrowserSync/browser-sync). These dependencies need to be updated together. Updates `serve-static` from 1.13.2 to 1.16.2 - [Release notes](https://github.com/expressjs/serve-static/releases) - [Changelog](https://github.com/expressjs/serve-static/blob/v1.16.2/HISTORY.md) - [Commits](expressjs/serve-static@v1.13.2...v1.16.2) Updates `browser-sync` from 2.29.3 to 3.0.3 - [Release notes](https://github.com/BrowserSync/browser-sync/releases) - [Changelog](https://github.com/BrowserSync/browser-sync/blob/master/CHANGELOG.md) - [Commits](BrowserSync/browser-sync@v2.29.3...v3.0.3) --- updated-dependencies: - dependency-name: serve-static dependency-type: indirect - dependency-name: browser-sync dependency-type: direct:development ... Signed-off-by: dependabot[bot] <[email protected]>

Bumps [rexml](https://github.com/ruby/rexml) from 3.3.7 to 3.3.9. - [Release notes](https://github.com/ruby/rexml/releases) - [Changelog](https://github.com/ruby/rexml/blob/master/NEWS.md) - [Commits](ruby/rexml@v3.3.7...v3.3.9) --- updated-dependencies: - dependency-name: rexml dependency-type: indirect ... Signed-off-by: dependabot[bot] <[email protected]>

Development

…/evoks/theme/static_src/multi-7fc3e3eb0d Bump serve-static and browser-sync in /evoks/theme/static_src

…/rexml-3.3.9 Bump rexml from 3.3.7 to 3.3.9 in /docs

…/evoks/theme/static_src/multi-659a406dfd Bump send and browser-sync in /evoks/theme/static_src

coderabbitai · 2025-10-27T08:55:10Z

Walkthrough

Two configuration changes were introduced: an RDF triple addition to set showTopConcepts to True for vocabulary URIs in Skosmos vocabulary construction, and a browser-sync devDependency version bump from ^2.26.14 to ^3.0.3.

Changes

Cohort / File(s)	Change Summary
RDF Configuration `evoks/Skosmos/skosmos.py`	Added RDF triple to set showTopConcepts to True for vocabulary URI in add_vocabulary method
Dependency Updates `evoks/theme/static_src/package.json`	Updated devDependency browser-sync from ^2.26.14 to ^3.0.3

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Both changes are straightforward configuration additions with no control flow or logic modifications—a single RDF triple addition and a routine dependency version bump.

Pre-merge checks and finishing touches

❌ Failed checks (1 warning, 1 inconclusive)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	You can run `@coderabbitai generate docstrings` to improve docstring coverage.
Title Check	❓ Inconclusive	The pull request title "Hierarchy" is a single, non-descriptive word that lacks specificity about the actual changes made. While the word might tangentially relate to vocabulary hierarchy concepts (given the `showTopConcepts` addition), it fails to clearly communicate what was changed or why. A developer scanning the pull request history would not understand that this PR adds `showTopConcepts` configuration to vocabularies and updates a development dependency. The title is too vague to serve its purpose of summarizing the main change for quick comprehension.	Consider revising the title to be more descriptive and specific, such as "Add showTopConcepts vocabulary configuration" or "Configure vocabulary hierarchy display settings". This would clearly communicate the primary purpose of the changeset and help team members understand the PR intent at a glance.

✅ Passed checks (1 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.

✨ Finishing touches

📝 Generate docstrings

🧪 Generate unit tests (beta)

Create PR with unit tests
Post copyable unit tests in a comment
Commit unit tests in branch hierarchy

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 49545ab and 7bf1176.

⛔ Files ignored due to path filters (2)

docs/Gemfile.lock is excluded by !**/*.lock
evoks/theme/static_src/package-lock.json is excluded by !**/package-lock.json

📒 Files selected for processing (2)

evoks/Skosmos/skosmos.py (1 hunks)
evoks/theme/static_src/package.json (1 hunks)

⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)

GitHub Check: test-all

🔇 Additional comments (2)

evoks/Skosmos/skosmos.py (1)

64-65: LGTM! Addition enables top concepts display.

The new RDF triple correctly sets showTopConcepts to True, following the same pattern as the existing fullAlphabeticalIndex configuration. This aligns with the PR's "Hierarchy" objective and will enable the display of top-level concepts in the Skosmos vocabulary interface.

evoks/theme/static_src/package.json (1)

24-24: No breaking changes to the public API in browser-sync v3.0.3.

Providing you haven't accessed any internal properties, everything will just work as there are no breaking changes to the public API. Internally, the project uses an immutable data structure for storing/retrieving options. The version is stable, non-deprecated, and has no security vulnerabilities. Standard configurations in bs.config.js will work as expected without modification.

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

codecov · 2025-10-27T08:59:23Z

Codecov Report

❌ Patch coverage is 79.64072% with 34 lines in your changes missing coverage. Please review.
✅ Project coverage is 92.36%. Comparing base (d53ae02) to head (7bf1176).
⚠️ Report is 165 commits behind head on development.

Files with missing lines	Patch %	Lines
evoks/evoks/middleware.py	50.00%	26 Missing ⚠️
evoks/evoks/views.py	61.53%	5 Missing ⚠️
evoks/Term/models.py	90.90%	2 Missing ⚠️
evoks/evoks/settings.py	95.65%	1 Missing ⚠️

Additional details and impacted files

@@               Coverage Diff               @@
##           development     #288      +/-   ##
===============================================
- Coverage        93.84%   92.36%   -1.49%     
===============================================
  Files               49       50       +1     
  Lines             1772     1846      +74     
===============================================
+ Hits              1663     1705      +42     
- Misses             109      141      +32

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

dependabot bot and others added 8 commits October 14, 2024 11:27

Merge pull request #278 from kit-data-manager/development

ce00cdb

Development

Merge pull request #199 from kit-data-manager/dependabot/npm_and_yarn…

d5d97e4

…/evoks/theme/static_src/multi-7fc3e3eb0d Bump serve-static and browser-sync in /evoks/theme/static_src

Merge pull request #230 from kit-data-manager/dependabot/bundler/docs…

9fa036c

…/rexml-3.3.9 Bump rexml from 3.3.7 to 3.3.9 in /docs

Merge pull request #216 from kit-data-manager/dependabot/npm_and_yarn…

fe2963a

…/evoks/theme/static_src/multi-659a406dfd Bump send and browser-sync in /evoks/theme/static_src

Show top concepts in skosmos hierarchy by default

7bf1176

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Hierarchy #288

Hierarchy #288

FelixFrizzy commented Oct 27, 2025 •

edited by coderabbitai bot

Loading

Uh oh!

coderabbitai bot commented Oct 27, 2025 •

edited

Loading

Uh oh!

codecov bot commented Oct 27, 2025

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Hierarchy #288

Are you sure you want to change the base?

Hierarchy #288

Conversation

FelixFrizzy commented Oct 27, 2025 • edited by coderabbitai bot Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary by CodeRabbit

Uh oh!

coderabbitai bot commented Oct 27, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Walkthrough

Changes

Estimated code review effort

Pre-merge checks and finishing touches

Uh oh!

codecov bot commented Oct 27, 2025

Codecov Report

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

FelixFrizzy commented Oct 27, 2025 •

edited by coderabbitai bot

Loading

coderabbitai bot commented Oct 27, 2025 •

edited

Loading